Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[stable7] fix(NcAppContent): Set normal scrollbar width on resizeable NcAppContentList #4747

Merged
merged 1 commit into from
Nov 2, 2023
Merged

[stable7] fix(NcAppContent): Set normal scrollbar width on resizeable NcAppContentList #4747

merged 1 commit into from
Nov 2, 2023

Conversation

mejo-
Copy link
Contributor

@mejo- mejo- commented Nov 2, 2023

Manual backport of #4710

When NcAppContentList is resizeable, a thin scrollbar is hard to catch with the mouse. Set scrollbar width to auto in this case.

Fixes: #4706

🏁 Checklist

  • ⛑️ Tests are included or are not applicable
  • 📘 Component documentation has been extended, updated or is not applicable

@mejo-
fix(NcAppContent): Set normal scrollbar width on resizeable NcAppCont…
6d88d70 
…entList

When `NcAppContentList` is resizeable, a thin scrollbar is hard to catch
with the mouse. Set scrollbar width to auto in this case.

Fixes: #4706

Signed-off-by: Jonas <[email protected]>
@mejo- mejo- added bug Something isn't working 3. to review Waiting for reviews design Design, UX, interface and interaction design accessibility Making sure we design for the widest range of people possible, including those who have disabilities feature: app-content-list Related to the app-content-list component labels Nov 2, 2023
@mejo- mejo- merged commit 1c2f926 into stable7 Nov 2, 2023
15 checks passed
@mejo- mejo- deleted the backport/4710/stable7 branch November 2, 2023 10:12
@nickvergessen nickvergessen mentioned this pull request Nov 7, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@szaimen szaimen szaimen approved these changes

@susnux susnux Awaiting requested review from susnux

@raimund-schluessler raimund-schluessler Awaiting requested review from raimund-schluessler

Assignees
No one assigned
Labels
3. to review Waiting for reviews accessibility Making sure we design for the widest range of people possible, including those who have disabilities bug Something isn't working design Design, UX, interface and interaction design feature: app-content-list Related to the app-content-list component
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@mejo- @szaimen